Application: Do not reuse Ctrl+F1 for help-overlay

It just gets overridden to do something else, as the 1st GtkWidget’s
class_init() adds it as a binding to toggle tooltips with the keyboard.
The last entity to hook it (& return TRUE) is the only one who gets it.

So, worse: If users needed to manually set accels for a ShortcutsWindow,
coincidentally waited until after the 1st GtkWidget init() to do it, &
copied what GtkApplication said in an attempt to be good & consistent —
they inadvertently broke the keyboard tooltips (except for users with a
GDK_KEY_KP_F1 – whatever that is) as their handler blocked GtkWidget’s.

So, one side has to drop this accelerator, and it seems clear that being
able to open tooltips with the keyboard is more important than having a
second accel for the help-overlay. We can make up a replacement later.

https://gitlab.gnome.org/GNOME/gtk/issues/626

entry completion: Avoid critical warnings

We must not call move-to-rect unless we have
a transient parent.

Merge branch 'fix-shifted-scrolling' into 'master'

gtkscrolledwindow: Consider shift key presses when decelerating

Closes #770

See merge request GNOME/gtk!286

gesturestylus: Add missing array g-i annotation for *_get_axes params

Update POTFILES.in

Merge branch 'kill-subsurface' into 'master'

Kill subsurfaces

See merge request GNOME/gtk!299

Move tools to a subdirectory

A small step towards splitting up gtk/

gdk: Drop GDK_SURFACE_SUBSURFACE

None of the backends supports this type anymore.

wayland: drop subsurfaces

We don't use them in GTK+ anymore.

Drop gtk_window_set_use_subsurface

This is no longer used.

entry completion: stop using subsurfaces

Instead, use a popup and gdk_surface_move_to_rect.
I have not tried to reproduce all details of the old
positioning logic, but moving the popup above/below
the entry works as before.

tooltip: Implement positioning using gdk_window_move_to_rect()

In order to make tooltip positioning portable, make use of the
move_to_rect API. Some semantical changes are made, as identical
semantics cannot be implemented using the move-to-rect API.

Primarily the implemented semantics are:

Position the tooltip in the center pixels slightly below (defaults to 4
units below) the tooltipped widget. This is always the case for keyboard
driven tooltips; the case where it tries to avoid the pointer cursor is
not implemented.

For pointer position triggered tooltips, implement the following
additional semantics:

Use the current cursor size to determine the padding used to enlarge the
anchor rectangle. This is to try to avoid the cursor overlapping the
tooltip.

If the anchor rectangle is too tall (meaning if we'd be constrained
and flip on the Y axis, it'd flip too far away from the originally
intended position), rely only on the pointer position to position the
tooltip. The approximate pointer cursor rectangle is used as a anchor
rectangle. Ideally we should use the actual pointer cursor rectangle
(image used as well as hotspot coordinate), but we don't have API to
get that information.

If the anchor rectangle isn't to tall, just make sure the tooltip isn't
too far away from the pointer position on the X axis.

Closes: #134
Closes: #432
Closes: #574
Closes: #579
Closes: #878

gdk/wayland: Avoid idempotent wl_subsurface.set_position calls

These may not result on wl_surface.frame callbacks, yet we do trigger
a frame clock tick that would get stuck on the lack of such callback.

https://bugzilla.gnome.org/show_bug.cgi?id=784314
https://gitlab.gnome.org/GNOME/gtk/issues/844

Closes: #844

gdk/wayland: Always map windows 'moved-to-rect' as popups

Only popups can make use of the move-to-rect semantics so it makes no
sense to try anything other surface type.

gdk/wayland: Don't remap non-subsurface as subsurface

Let's just use the fact that a window was mapped as a subsurface to
remap it above another transient parent instead of relying on the more
complicated 'should-map-as-subsurface' helper function.

Merge branch 'win32-smooth-scrolling-gtk4' into 'master'

GDK W32: Support smooth scrolling (GTK4)

See merge request GNOME/gtk!298

menu: Remove key_controller member

Unused.

menu: Remove initially_pushed_in flag

Unused.

menu: Remove seen_item_enter flag

Unused.

menu: Remove have_position flag

It's always FALSE.

menu: Remove an unnecessary gtk_widget_destroy() call

The unref will already properly free the menu item's resources.

GDK W32: Support smooth scrolling

Set delta_x or delta_y for GdkScrollEvent.
HIWORD (wParam) in WM_MOUSE(H)WHEEL is the scroll delta.
A delta value of WHEEL_DELTA (which is 120) means scrolling
one full unit of something (for example, a line).

The delta should also be multiplied by the value that the
SystemParametersInfo (SPI_GETWHEELSCROLL(LINES|CHARS), 0, &value, 0)
call gives back, unless it gives back 0xffffffff, in which case
it indicates that scrolling is page- or screen-based, not line-based
(GDK doesn't support that at the moment).

Also, all deltas should be inverted, since MS sends negative deltas
when scrolling down (rotating the wheel back, in the direction of
the user).

With deltas set the mode should be set to GDK_SCROLL_SMOOTH.

Fixes issue 1263.

Don't fail the build if c++ isn't found

The intention of this check was to skip the keyword
test if no c++ compiler is found. But the meson
docs say that add_languages() will abort unless we
pass required: false.

Merge branch 'fix-shell-colorpicker' into 'master'

colorpickershell: Unpack the tuple returned from PickColor()

See merge request GNOME/gtk!296

print portal: don't leak window handles

Pointed out by Christian Persch.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1274

print portal: Plug a memory leak

Pointed out by Christian Persch.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1271

Use defines for bus names

This helps eradicate typos (we had some here).

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1267

Merge branch 'sort' into 'master'

updateiconcache: Sort list of entries

See merge request GNOME/gtk!297

font chooser: Remove an unneeded ifdef

We can just always create the axes hashtable.

updateiconcache: Sort list of entries

because filesystem readdir order is indeterministic.

Without this patch, building openSUSE's balsa package
had variations between builds in /usr/share/balsa/icon-theme.cache

Merge branch 'master' into 'master'

window: Fix memory leak

See merge request GNOME/gtk!295

colorpickershell: Unpack the tuple returned from PickColor()

When calling PickColor on org.gnome.Shell, we get back an "a{sv}", which
GDBus provides to us as "(a{sv})".

At the minute we're not unpacking this tuple, and so picking fails with
messages like:

  GLib-CRITICAL **: 13:38:19.439: g_variant_lookup_value: assertion 'g_variant_is_of_type (dictionary, G_VARIANT_TYPE ("a{s*}")) || g_variant_is_of_type (dictionary, G_VARIANT_TYPE ("a{o*}"))' failed

  Gtk-WARNING **: 13:38:19.439: Picking color failed: No color received

Let's unpack it.

window: Fix memory leak

https://gitlab.gnome.org/GNOME/gtk/issues/1268

Update Turkish translation

Update Polish translation

Merge branch 'gtk-4-improve-error-message' into 'master'

gtkplacessidebar: Improve error message when unlocking volume fails

See merge request GNOME/gtk!267

Update Polish translation

migration guide: Mention GtkEventBox

Its gone.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1243

README: Fix Gdk doc website link

README: Document nightly docs location

Useful for exposing the documentation early.

gtkplacessidebar: Improve error message when unlocking volume fails

Updated Lithuanian translation

widget: Rename width and height member to {width,height}_request

Because that's much closer to the meaning of those values.

main: Pass toplevel_widget directly to GtkWidget API

We already have the toplevel as GtkWidget: toplevel_widget.

sizerequestcache: Inline out-var assignments

The additional assignment to the old result variable just adds an
indirection even though we know the point where we assign it in all
cases. Just pass the values out and return in those cases instead.

widgetprivate: Remove useless prototype

We implement that function lower down in that header file.

pathbar: Remove priv pointer

menuitem: Don't redraw unnecessarily

The gtk_widget_(un)set_state_flags calls before will do this if
necessary.

menu: Use correct content height

menu: Allocate children according to scroll_offset

This was done by simply moving the bin_window before but now we have to
do it ourselves.

levelbar: Don't return a double value from a gboolean function

acitonbar: Remove snapshot implementation

modelbutton: Remove snapshot implementation

Previously, GtkBin was only snapshot'ing its one and only child, but
nowadays it doesn't implement snapshot at all and the default
implementation in GtkWidget just snapshots all child widgets, which is
exactly what the implementation in gtkmodelbutton.c was doing.

Make color picker circular

The need for center alignment was pointed out by Timm Baeder.

gtkscrolledwindow: Consider shift key presses when decelerating

Otherwise horizontal scrolling using the shift key would decelerate
vertically.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/770

Update German translation

Merge branch 'sidebar-spinner-visibility' into 'master'

sidebarrow: Hide busy_spinner by default

See merge request GNOME/gtk!285

sidebarrow: Hide busy_spinner by default

Since the original implementation was likely based on GTK+ 3, the change
in default visibility might have not been considered, which results in
all rows suddenly sporting a visible spinner when opening a fresh file
chooser.

Merge branch 'fix-app-id' into 'master'

Make xdg-shell's app_id match GApplication's application-id property

Closes #653

See merge request GNOME/gtk!284

color picker: Also try kwin

use a kwin color picker when we run under kwin.

Only use the portal color picker when appropriate

This is just moving this check around.

Add a kwin color picker

This uses the org.kde.kwin.ColorPicker interface.

Merge branch 'gbsneto/issue-1258' into 'master'

listbox: Unparent child after removing from sequence

Closes #1258

See merge request GNOME/gtk!283

listbox: Store child iter in a variable when removing

Unparenting a GtkListBoxRow can drop its last reference, which
will free its memory. Right after unparenting, though, we were
accessing the row's iter - which assumes that the row is still
alive. This causes a crash when, for example, binding two or
more models to the listbox.

Fix that by storing the iter in a variable, and not trying to
access it after unparenting. After unparenting, the variables
that are potentially garbage were explicitly assigned NULL for
clarity.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1258

demos: Make desktop file names match the application id

examples: Use appliction id as desktop file base name

docs: Explain desktop file id == application_id in migration guide

gdk: Use application_id as xdg_shell's app_id if available

This (manually) reverts commit e1fd87728dd841cf1d71025983107765e395b152

This makes sure xdg_shell's app_id matches the DBus name of the the
appliation when using GtkApplication. It also allows the compositor to
derive the desktop file ID from the app_id:

   https://cgit.freedesktop.org/wayland/wayland-protocols/tree/stable/xdg-shell/xdg-shell.xml?id=298d888ac718eae57ff2245d373d4327074506ea#n598

Closes #653

container: Remove focus chains

They're prectically unused.

Widgets who want to influcence the focus chain behavior should implement
GtkWidgetClass::focus and deal with focus there.

widgetfocus: Fix copy/paste error

This lead to erratic focus behavior with <Tab> in hboxes, where the
smallest child got focus first instead of the leftmost child.

colorpicker: Fix gcc warning

Mention gtk_widget_show_all in the migration guide

This function is gone.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1242

Merge branch 'issue1214-tooltips-win32-gtk4' into 'master'

Gdk-Win32: Correct handling of transient state changes

See merge request GNOME/gtk!280

Gdk-Win32: Correct handling of transient state changes

See merge request !248

bindings: Remove GTK_TYPE_IDENTIFIER

bindings now treat identifiers and strings the same way.

The only difference was that one allowed lookup of enum/flags by name
while the other didn't and g_warning()ed. Now both work.

color picker: Better debug spew

This will help in debugging issues.

Updated Czech translation

gdk: Fix Windows and OS X backend for keyevent.string removal

treeview: Fix compiler warning

gdk: Remove key_event->string and key_event->length

They're unused.

gdk: Remove gdk_event_get_string()

You want to use an IM module to get strings out of keypresses, not some
crude hack that only works on X11 and Wayland anyway.

Update POTFILES.skip

Updated Czech translation

colorpickerportal: Fix memory leak

inspector: Remove unnecessary cast

Pointed out by Timm.

Merge branch 'gtk-4-add-spinner-to-sidebar-row' into 'master'

gtkplacessidebar: Show busy spinner in sidebar row during mount ops

See merge request GNOME/gtk!261

Merge branch 'wip/carlosg/event-final-cleanup' into 'master'

Wip/carlosg/event final cleanup

See merge request GNOME/gtk!279

tests: Port testwidgetfocus to using a motion controller

demos: Update changedisplay demo to using GtkGesture

tests: Port motion-compression to using a motion controller

gtkscrolledwindow: Use controller for motion capturing

Perform scrollbar visibility checks through a motion controller,
always based on GtkScrolledView-relative coordinates. The captured
event handler remains though, for a tiny bit of GDK_SCROLL event
handling.

gtkwindow: Replace captured event handler with motion controller

testsuite: Update window test to use gestures when interactive

tests: Use gesture in testtreepos

tests: Use gesture to popup menu in testiconview

tests: Update testgrid to use gestures

demos: Update hypertext demo to use controllers

demos: Update search_entry2 demo to use key capturing API

No more GdkEvent juggling.

demos: drop "Event axes" demo

It's too lowlevel to be practical anymore with gtk4, and there's
the Paint and Gestures demos to demonstrate input capabilities
better than this.

gtk-demo: Add pad controller handling to "Paint" demo

So it's more lifelike.